Skip to content

[Add] coverage for extend extension methods#85

Open
samatstariongroup wants to merge 1 commit intodevelopmentfrom
implement-tests-for-extend-methods
Open

[Add] coverage for extend extension methods#85
samatstariongroup wants to merge 1 commit intodevelopmentfrom
implement-tests-for-extend-methods

Conversation

@samatstariongroup
Copy link
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the SysML2.NET code style guidelines
  • I have provided test coverage for my change (where applicable)

Description

Adds unit tests for all extension methods used to compute the value of derived properties. As these are implemented the tests will be replaced by actual behavior tests and not just verifying that the implementation is not yet there. So, in the end raising coverage knowing its not yet complete and having placeholders to verify the extension methods during implementation time.

Copy link
Collaborator

@antoineatstariongroup antoineatstariongroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Core content is fine.
The usage of Assert.Throws could be move to the more readable and fluent Assert.That syntax

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments